Skip to content

Conversation

@mururu
Copy link
Contributor

@mururu mururu commented Apr 8, 2025

There were no periods elsewhere, I aligned everything by removing the period.

Copilot AI review requested due to automatic review settings April 8, 2025 02:31
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

Copy link
Collaborator

@juruen juruen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for helping to make the project more consistent!

@juruen juruen merged commit 973fb5d into github:main Apr 8, 2025
9 checks passed
manian0430 pushed a commit to ChrisLally/github-mcp-server that referenced this pull request Apr 12, 2025
DaleSeo pushed a commit to DaleSeo/github-mcp-server that referenced this pull request Oct 24, 2025
… models (github#170)

* Add ArXiv MCP Server for academic research and paper analysis

Features:
• Search arXiv papers with advanced filtering (date, category, query)
• Download and store papers locally as markdown
• Read and analyze paper content programmatically
• Deep research analysis prompts for comprehensive paper review
• Configurable local storage path for paper management

Technical Details:
- Category: search (academic research focused)
- License: Apache 2.0 (compatible with Docker registry)
- Source: https://github.com/jasonleinart/arxiv-mcp-server
- Build Status: Successfully tested and validated
- Docker Image: mcp/arxiv-mcp-server

Target Users:
Perfect for researchers, academics, and AI assistants conducting
literature reviews, research analysis, and academic paper exploration.

Differentiates from existing paper-search server by providing:
- ArXiv-specific optimizations and direct API integration
- Local paper storage and management capabilities
- Research-focused analysis prompts and workflows
- Advanced filtering and categorization options

* FIX: Add proper volume mounting to ArXiv MCP server

- Maps user storage path to /app/papers inside container
- Enables file persistence for downloaded papers
- Fixes ephemeral container issue where files weren't saved
- Follows pattern used by kubectl-mcp-server and elevenlabs
- Essential for Docker MCP Toolkit volume mounting

* FIX: Use environment variable for volume mounting

- Change from parameter template to environment variable approach
- Docker MCP Toolkit processes  but not {{parameter}} templates
- Maps storage_path parameter to STORAGE_PATH env var for volume mounting
- Should resolve volume mounting issue where files weren't persisting

* FIX: Correct volume mounting configuration per PR feedback

- Change volume mapping from $STORAGE_PATH to {{arxiv-mcp-server.storage_path}} template syntax
- Add ARXIV_STORAGE_PATH environment variable for internal container path
- Fixes volume mounting issue identified by @cmrigney in PR review
- Follows Docker MCP Registry configuration standards per docs/configuration.md

* FIX: Apply reviewer's suggestion - Use only ARXIV_STORAGE_PATH environment variable

- Remove redundant STORAGE_PATH environment variable as suggested by @cmrigney
- Keep only ARXIV_STORAGE_PATH with '/app/papers' value for container internal path
- Simplifies configuration while maintaining full functionality
- Volume mounting handles path mapping: {{arxiv-mcp-server.storage_path}}:/app/papers
- ArXiv server auto-detects Docker environment and uses correct storage path

* Update ArXiv MCP Server with enhanced tool descriptions for local AI models

- Add enhanced tool descriptions optimized for local AI model compatibility
- Improve Docker MCP Gateway integration with detailed context
- Address community feedback about sparse tool descriptions causing confusion
- Add tags for better discoverability: local-models, docker-gateway, enhanced-descriptions

This update resolves issues where local AI models (Llama, Mistral, etc.) were
getting confused by minimal tool descriptions when using Docker MCP Gateway.

Fixes: docker/mcp-gateway#93
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants